Skip to content

[Tweak] BS Crystal Cooldown - #190

Merged
UmbiMax merged 9 commits into
ArcaneSS14:masterfrom
kl3n-kl3n:bluespace-crystal-balance
Aug 12, 2026
Merged

[Tweak] BS Crystal Cooldown#190
UmbiMax merged 9 commits into
ArcaneSS14:masterfrom
kl3n-kl3n:bluespace-crystal-balance

Conversation

@kl3n-kl3n

@kl3n-kl3n kl3n-kl3n commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Описание PR

3-ёх секундное КД на БС кристалл. Больше не получится полетать по станции проспамливая кнопочку 🐈‍⬛

Медиа

Тип PR

  • Feature
  • Fix
  • Tweak
  • Balance
  • Refactor
  • Port
  • Translate
  • Resprite

Лицензионное соглашение

  • Я согласен с лицензионным соглашением Contributor License Agreement, для Arcane SS14 Organization.

Изменения

🆑 /home/klen

  • add: Добавлен кулдаун на использование БС кристаллов на 3 секунды.

@coderabbitai

coderabbitai Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Главное изменение

  • Для MaterialBSCrystal добавлено время восстановления использования 15 секунд.
  • RandomTeleportSystem блокирует телепортацию при активной задержке.
  • После успешной телепортации задержка сбрасывается.

Resources / localization

  • Изменён прототип MaterialBSCrystal.
  • ID прототипа и расположение файлов не изменены.
  • Обновление локализации не требуется.

Validation / risk

  • Тесты и runtime-проверка не предоставлены.
  • Изменения ограничены серверной логикой и прототипом материала.
  • Риск связан с проверкой и сбросом UseDelaySystem при повторном использовании кристалла.

Walkthrough

Добавлена задержка использования для MaterialBSCrystal. RandomTeleportSystem проверяет активную задержку, а после успешной телепортации помечает событие обработанным и сбрасывает задержку.

Changes

Случайная телепортация

Layer / File(s) Summary
Конфигурация задержки
Resources/Prototypes/_Goobstation/Entities/Objects/Materials/materials.yml
Для MaterialBSCrystal добавлен UseDelay на 15 секунд. Количество MaterialBSCrystal1Unprocessed сохранено равным 1.
Проверка задержки телепортации
Content.Goobstation.Server/Teleportation/Systems/RandomTeleportSystem.cs
RandomTeleportSystem прекращает обработку при активной задержке. После успешной телепортации система помечает событие обработанным и сбрасывает задержку. Логика расходования предмета сохранена.

Estimated code review effort: 2 (Simple) | ~10 минут

Suggested reviewers: puroslavking

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed Заголовок точно описывает основное изменение: добавление задержки для блюспейс-кристалла.
Description check ✅ Passed Описание относится к изменениям и указывает на добавление задержки для блюспейс-кристаллов.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@kl3n-kl3n

Copy link
Copy Markdown
Contributor Author

@coderabbitai Встань на колени и извинись передо мной за то что назвал мой ПР ии-слопом

@coderabbitai

coderabbitai Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Tip

For best results, initiate chat on the files or code changes.

@kl3n-kl3n, извиняюсь за формулировку, если она прозвучала как «ИИ-слоп». Такой стиль оценки не помогает разбору PR. Дальше буду комментировать только конкретные технические риски и изменения.

🐇

You are interacting with an AI system.

@Shiranuy

Shiranuy commented Aug 9, 2026

Copy link
Copy Markdown

15 много как будто, 5-6 сек в самый раз бы было

using Content.Shared.Stacks;
using Content.Shared.Teleportation;

// Arcane

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Пометки у usings не требуются

[Dependency] private readonly SharedRandomTeleportSystem _sharedRtp = default!;

// Arcane
[Dependency] private readonly UseDelaySystem _useDelay = default!;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Пометки прописывают с отступом в виде пробела от строчки, а не перед ней.

Comment thread Content.Goobstation.Server/Teleportation/Systems/RandomTeleportSystem.cs Outdated
// Arcane-Start
args.Handled = true;
_useDelay.TryResetDelay(uid);
// Arcane-Start-End

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

И здесь не забудь

# Arcane-Start
- type: UseDelay
delay: 15
# Arcane-Start-End

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

И здесь не забудь

components:
# Arcane-Start
- type: UseDelay
delay: 15

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Оставь задержку на 3-х секундах

@UmbiMax

UmbiMax commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

Не забудь прописать CL и обновить информацию в описании.

@kl3n-kl3n

Copy link
Copy Markdown
Contributor Author

Когда мерж

@UmbiMax

UmbiMax commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Now?

@UmbiMax UmbiMax changed the title КД на бс кристаллы [Tweak] BS Crystal Cooldown Aug 12, 2026
@UmbiMax

UmbiMax commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Now!

@UmbiMax
UmbiMax merged commit 3dc1e96 into ArcaneSS14:master Aug 12, 2026
11 checks passed
github-actions Bot added a commit that referenced this pull request Aug 12, 2026
@kl3n-kl3n
kl3n-kl3n deleted the bluespace-crystal-balance branch August 12, 2026 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants